Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multitool can be used to find the area's APC #16544

Merged
merged 5 commits into from
Sep 17, 2024

Conversation

Runian
Copy link
Contributor

@Runian Runian commented Sep 13, 2024

About The Pull Request

Ported from cmss13-devs/cmss13#7020

Having at least one point in engineering lets you use a multitool in hand which helps you find the APC in the area.
image

Why It's Good For The Game

There is this one intel computer that no one ever does in the streets of LV759 because the APC for it must be in the 5th dimension. Since I can't find it, I need to code something to help me (and everyone else) find it.

... And it also helps new engineers do their job by finding the broken APC that they need to fix for objectives.

Changelog

🆑
add: If you have any engineering skills, you can use a multitool will help you find the APC of the room you're in.
/:cl:

@tgstation-server tgstation-server added the Feature New interesting mechanics with new interesting bugs label Sep 13, 2024
@AusarAtkin
Copy link
Contributor

Speed merge this shit , for fellow visually imparied engis.

@iforgotmeword
Copy link

iforgotmeword commented Sep 13, 2024

yes but make it have a cast time if you are not a eng instead

I don't want CBT (Combat Buddy Torture, WHY THE FUCK ARE YOU SO DUMB? JUST USE THE MULTITOOL BUDDY PLEASE)

@SimplySmiley123
Copy link

yes but make it have a cast time if you are not a eng instead

I don't want CBT (Combat Buddy Torture, WHY THE FUCK ARE YOU SO DUMB? JUST USE THE MULTITOOL BUDDY PLEASE)

Yeah, honestly, would be nice.

Either way the PR is already amazing and I want it for my visually impaired ass.

code/game/objects/items/multitool.dm Outdated Show resolved Hide resolved
Comment on lines 55 to 56

/obj/item/tool/multitool/proc/remove_apc_highlight(client/user_client, image/highlight_image)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

autodoc pls

@ivanmixo ivanmixo merged commit a92eff7 into tgstation:master Sep 17, 2024
39 checks passed
github-actions bot added a commit that referenced this pull request Sep 17, 2024
@Runian Runian deleted the multitool-locate-apc branch September 17, 2024 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New interesting mechanics with new interesting bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants